tests: drivers: flash: common: run on SDP MSPI #20542

jaz1-nordic · 2025-02-20T15:47:15Z

No description provided.

github-actions · 2025-02-20T15:47:42Z

After documentation is built, you will find the preview for this PR here.

NordicBuilder · 2025-02-20T15:48:35Z

The following west manifest projects have changed revision in this Pull Request:

Name	Old Revision	New Revision	Diff

✅ All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

NordicBuilder · 2025-02-20T15:50:46Z

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 18

Inputs:

Sources:

sdk-nrf: PR head: 497433f00625c0b879ddde5d427f44dacee28cac

more details

sdk-nrf:

PR head: 497433f00625c0b879ddde5d427f44dacee28cac
merge base: 2a24ce429465dca6c3b56474f0a15911988615db
target head (main): 42a7901de8753fa94fb4472b363d395b28893a24
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (13)

.github
│  │ test-spec.yml
CODEOWNERS
applications
│  ├── sdp
│  │  ├── mspi
│  │  │  ├── Kconfig
│  │  │  ├── boards
│  │  │  │  │ nrf54l15dk_nrf54l15_cpuflpr.conf
scripts
│  ├── ci
│  │  │ tags.yaml
snippets
│  ├── sdp
│  │  ├── mspi
│  │  │  ├── app.conf
│  │  │  ├── soc
│  │  │  │  │ nrf54l15_cpuapp.overlay
tests
│  ├── zephyr
│  │  ├── drivers
│  │  │  ├── flash
│  │  │  │  ├── common
│  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  ├── Kconfig
│  │  │  │  │  ├── prj.conf
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ main.c
│  │  │  │  │  ├── sysbuild.conf
│  │  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: acee3b0b2b
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:acee3b0b2b_bece0367df

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
- sdk-nrf test count: 40
✅ Integration tests
- ✅ test-low-level

Disabled integration tests

- desktop52_verification
- doc-internal
- test-fw-nrfconnect-apps
- test-fw-nrfconnect-ble_mesh
- test-fw-nrfconnect-ble_samples
- test-fw-nrfconnect-boot
- test-fw-nrfconnect-chip
- test-fw-nrfconnect-fem
- test-fw-nrfconnect-nfc
- test-fw-nrfconnect-nrf-iot_cloud
- test-fw-nrfconnect-nrf-iot_libmodem-nrf
- test-fw-nrfconnect-nrf-iot_lwm2m
- test-fw-nrfconnect-nrf-iot_mosh
- test-fw-nrfconnect-nrf-iot_positioning
- test-fw-nrfconnect-nrf-iot_samples
- test-fw-nrfconnect-nrf-iot_serial_lte_modem
- test-fw-nrfconnect-nrf-iot_thingy91
- test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- test-fw-nrfconnect-nrf_crypto
- test-fw-nrfconnect-proprietary_esb
- test-fw-nrfconnect-ps
- test-fw-nrfconnect-rpc
- test-fw-nrfconnect-rs
- test-fw-nrfconnect-tfm
- test-fw-nrfconnect-thread
- test-fw-nrfconnect-zigbee
- test-sdk-audio
- test-sdk-dfu
- test-sdk-find-my
- test-sdk-mcuboot
- test-sdk-pmic-samples
- test-sdk-sidewalk
- test-sdk-wifi
- test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

nordic-piks · 2025-02-21T05:55:59Z

New alt-config for zephyr/tests/drivers/flash/common/testcase.yaml added at CI.

github-actions · 2025-02-26T13:53:59Z

You can find the documentation preview for this PR here.

Increase RX buffer size to be able to run flash/common test. Signed-off-by: Jakub Zymelka <[email protected]>

Increase size of the PBUF RX to be able to handle longer messages. Signed-off-by: Jakub Zymelka <[email protected]>

nordic-piks · 2025-03-06T13:11:00Z

tests/zephyr/drivers/flash/common/testcase.yaml

Please add e.g nrf.extended. to the name, to it can be easily recognizable.

…ration Add local copy of test configuration to enable SDP MSPI flash common test. Signed-off-by: Jakub Zymelka <[email protected]>

Twister does not have a functionality to quarantine single test case. As a quick workaround, move source from zephyr, but disable failing test with ifdef. When issue is fixed, this commit should be reverted. Signed-off-by: Marcin Szymczyk <[email protected]>

Current memory configuration is SINGLE mode. We need to remove unused pins in this mode until SDP app supports it properly. Otherwise it causes problems with reset ans WP pins. Signed-off-by: Jakub Zymelka <[email protected]>

sonarqubecloud · 2025-03-06T14:03:26Z

Quality Gate passed

Issues
8 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

pasamdk

K

jaz1-nordic requested review from a team as code owners February 20, 2025 15:47

github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Feb 20, 2025

NordicBuilder added manifest-zephyr DNM labels Feb 20, 2025

nordic-segl approved these changes Feb 21, 2025

View reviewed changes

jaz1-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch from a599a79 to fb48824 Compare February 25, 2025 13:12

jaz1-nordic requested a review from a team as a code owner February 25, 2025 13:12

jaz1-nordic changed the title ~~drivers: sdp: mspi: fix for stuck in semaphore timeouts~~ tests: drivers: flash: common: run on SDP MSPI Feb 25, 2025

nordicjm approved these changes Feb 26, 2025

View reviewed changes

jaz1-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch from fb48824 to dba12eb Compare February 26, 2025 13:32

magp-nordic approved these changes Feb 26, 2025

View reviewed changes

jaz1-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch from dba12eb to ebd1854 Compare February 28, 2025 14:45

github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Feb 28, 2025

jaz1-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch from ebd1854 to c95c387 Compare February 28, 2025 14:54

masz-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch from c95c387 to e76f70c Compare March 5, 2025 15:29

github-actions bot removed the manifest label Mar 5, 2025

NordicBuilder removed manifest-zephyr DNM labels Mar 5, 2025

masz-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch 2 times, most recently from c696441 to 4d0b2db Compare March 5, 2025 16:55

nordic-segl approved these changes Mar 6, 2025

View reviewed changes

masz-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch 2 times, most recently from 3771dc1 to b9e589b Compare March 6, 2025 12:15

applications: sdp: mspi: Change RX buffer size

16ac742

Increase RX buffer size to be able to run flash/common test. Signed-off-by: Jakub Zymelka <[email protected]>

masz-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch from b9e589b to 190213d Compare March 6, 2025 12:19

sdp: mspi: Increase PBUF RX buffer size

d793aa5

Increase size of the PBUF RX to be able to handle longer messages. Signed-off-by: Jakub Zymelka <[email protected]>

masz-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch 2 times, most recently from aa663f6 to 68e4254 Compare March 6, 2025 12:26

nordic-piks reviewed Mar 6, 2025

View reviewed changes

jaz1-nordic and others added 3 commits March 6, 2025 14:58

tests: drivers: flash: common: add SDP MSPI flash common test configu…

1662994

…ration Add local copy of test configuration to enable SDP MSPI flash common test. Signed-off-by: Jakub Zymelka <[email protected]>

snippets: sdp: mspi: remove unused pins configuration

497433f

Current memory configuration is SINGLE mode. We need to remove unused pins in this mode until SDP app supports it properly. Otherwise it causes problems with reset ans WP pins. Signed-off-by: Jakub Zymelka <[email protected]>

masz-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch from 0962ce7 to 497433f Compare March 6, 2025 13:59

masz-nordic requested review from a team, PerMac and katgiadla as code owners March 6, 2025 13:59

masz-nordic added this to the 3.0.0 milestone Mar 6, 2025

masz-nordic requested a review from nordic-piks March 6, 2025 14:08

nordic-piks approved these changes Mar 7, 2025

View reviewed changes

nordic-segl approved these changes Mar 7, 2025

View reviewed changes

PerMac approved these changes Mar 7, 2025

View reviewed changes

PerMac merged commit 5b0c9b0 into nrfconnect:main Mar 7, 2025
18 checks passed

pasamdk mentioned this pull request Apr 26, 2025

K pasamdk/Test#6

Open

pasamdk reviewed Apr 26, 2025

View reviewed changes

jaz1-nordic deleted the nrfx-7100_sdp_mspi_flash_test_fail branch April 28, 2025 06:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

tests: drivers: flash: common: run on SDP MSPI #20542

tests: drivers: flash: common: run on SDP MSPI #20542

Uh oh!

jaz1-nordic commented Feb 20, 2025

Uh oh!

github-actions bot commented Feb 20, 2025

Uh oh!

NordicBuilder commented Feb 20, 2025 •

edited

Loading

Uh oh!

NordicBuilder commented Feb 20, 2025 •

edited

Loading

Uh oh!

nordic-piks commented Feb 21, 2025

Uh oh!

github-actions bot commented Feb 26, 2025

Uh oh!

nordic-piks Mar 6, 2025

Uh oh!

sonarqubecloud bot commented Mar 6, 2025

Uh oh!

Uh oh!

pasamdk left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

tests: drivers: flash: common: run on SDP MSPI #20542

tests: drivers: flash: common: run on SDP MSPI #20542

Uh oh!

Conversation

jaz1-nordic commented Feb 20, 2025

Uh oh!

github-actions bot commented Feb 20, 2025

Uh oh!

NordicBuilder commented Feb 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

NordicBuilder commented Feb 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

Uh oh!

nordic-piks commented Feb 21, 2025

Uh oh!

github-actions bot commented Feb 26, 2025

Uh oh!

nordic-piks Mar 6, 2025

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Mar 6, 2025

Quality Gate passed

Uh oh!

Uh oh!

pasamdk left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

NordicBuilder commented Feb 20, 2025 •

edited

Loading

NordicBuilder commented Feb 20, 2025 •

edited

Loading